Skip to content

Conversation

sBouzols
Copy link
Contributor

fix(SA,SE,CC): reset exportSuccessful states when context like study, node or rootNetwork change as well as resultType, computation type or tab change.

refactor(SE): move export CSV Button in a separeted component like for other computations

…y, node or rootNetwork change as well as resultType, computation type or tab change.

refactor(SE): move export CSV Button in a separeted component like for other computations

Signed-off-by: sBouzols <[email protected]>
Copy link
Contributor

@FranckLecuyer FranckLecuyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests:
no reset is done on sensi result column sort change
no reset is done when switching to another tab (tree or spreadsheet tab), and returning to the currently result tab

…t CSV button status

fix(SE): Add `sortConfig` trigger to reset export CSV button status, for other computation types results are invalidated when sorting then the button is disabled and trigger a status reset.

Signed-off-by: sBouzols <[email protected]>
@sBouzols
Copy link
Contributor Author

Tests: no reset is done on sensi result column sort change no reset is done when switching to another tab (tree or spreadsheet tab), and returning to the currently result tab

fixed, please check again

Copy link

@FranckLecuyer FranckLecuyer self-requested a review September 16, 2025 11:18
@souissimai
Copy link
Contributor

tested ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants